Skip to content

Show error when video has age limit and setting is disabled#3410

Merged
wb9688 merged 2 commits intoTeamNewPipe:devfrom
wb9688:fix-restricted
Apr 11, 2020
Merged

Show error when video has age limit and setting is disabled#3410
wb9688 merged 2 commits intoTeamNewPipe:devfrom
wb9688:fix-restricted

Conversation

@wb9688
Copy link
Member

@wb9688 wb9688 commented Apr 10, 2020

What is it?

  • Bug fix (user facing)
  • Feature (user facing)
  • Code base improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

Now it will show an error for videos with an age limit when the option to view age restricted videos is disabled in the settings:

Screenshot of video detail fragment with error

Apparently there already was a bug in NewPipe that it will still show the tabs when an error occured, and this PR doesn't fix that yet, though it makes it much more noticable.

Fixes the following issue(s)

This PR will fix #776.

Agreement

@wb9688 wb9688 marked this pull request as ready for review April 11, 2020 07:30
@wb9688 wb9688 requested a review from Stypox April 11, 2020 07:31
@wb9688 wb9688 merged commit b7ef60e into TeamNewPipe:dev Apr 11, 2020
@vnagel vnagel mentioned this pull request Apr 11, 2020
1 task
This was referenced Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Age restricted content option not working

3 participants